Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(nextjs): Don't report NEXT_REDIRECT from browser #14440

Merged

Conversation

NEKOYASAN
Copy link
Contributor

Fixes #14436

Reduces error spam by not reporting NEXT_NOT_FOUND error which get thrown when use the redirect() functions.

From ServerComponents were addressed in #7642 and ServerAction in #10474, but this PR addresses what is reported from the Client (Browser).

@NEKOYASAN
Copy link
Contributor Author

NEKOYASAN commented Nov 23, 2024

I've read the Contributing Guide and know that testing is required, but I don't know how to test this one.... sorry...

@mydea mydea requested review from lforst and chargome November 25, 2024 08:04
@lforst
Copy link
Member

lforst commented Nov 25, 2024

Thanks!

@chargome chargome merged commit 8f6dd04 into getsentry:develop Nov 25, 2024
45 checks passed
mydea pushed a commit that referenced this pull request Nov 25, 2024
This PR adds the external contributor to the CHANGELOG.md file, so that
they are credited for their contribution. See #14440

Co-authored-by: chargome <20254395+chargome@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NEXT_REDIRECT error to be reported as an Unhandled Error in Server Action redirect function.
3 participants